Skip to content
This repository has been archived by the owner on Nov 16, 2022. It is now read-only.

Refactor to support option by value #24 #26

Merged
merged 2 commits into from
Feb 19, 2019
Merged

Conversation

manico
Copy link
Member

@manico manico commented Feb 18, 2019

This PR added support to get color if returnType is value.
A lot of refactoring there so please check it, I need a couple of eyes more here.
Example is in App.vue.

Fix #24

@manico manico added the enhancement New feature or request label Feb 18, 2019
@manico manico self-assigned this Feb 18, 2019
@manico manico requested review from Sneki and Lanchi February 18, 2019 10:45
@codecov
Copy link

codecov bot commented Feb 18, 2019

Codecov Report

Merging #26 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master     #26   +/-   ##
======================================
  Coverage    7.69%   7.69%           
======================================
  Files           3       3           
  Lines          13      13           
======================================
  Hits            1       1           
  Misses         12      12

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7b63be8...d8036fe. Read the comment docs.

@manico manico mentioned this pull request Feb 18, 2019
@Lanchi
Copy link
Member

Lanchi commented Feb 18, 2019

screenshot 2019-02-18 at 11 58 26

Shouldn't input always show color name? This happens when `returnType` is `value`. We had something like this already reported in chmjs/chameleon-bundle-seven/issues/32

@manico
Copy link
Member Author

manico commented Feb 18, 2019

I can set it to name if color is found, and on value if not.
Good catch, will change.

Copy link
Member

@Lanchi Lanchi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good now.

@manico manico merged commit ca6da1a into master Feb 19, 2019
@manico manico deleted the feature/colorByValue branch February 19, 2019 13:24
@manico
Copy link
Member Author

manico commented Feb 19, 2019

@Sneki Please just tag and push new version of the package and try it out in Builder.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants